home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 3491 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  2.2 KB
  1. Path: news.infi.net!usenet
  2. From: nngis@norfolk.infi.net (Greg DiGiorgio)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: New C Programmer Has A Problem
  5. Date: 29 Jan 1996 13:28:15 GMT
  6. Organization: Customer of InfiNet
  7. Message-ID: <4eii1f$5il@nw002.infi.net>
  8. References: <4ehpa3$6kl@nntp.novia.net>
  9. Reply-To: nngis@norfolk.infi.net
  10. NNTP-Posting-Host: h-skeletoncrew.norfolk.infi.net
  11. Mime-Version: 1.0
  12. X-Newsreader: WinVN 0.99.3
  13.  
  14. In article <4ehpa3$6kl@nntp.novia.net>, tsyslo@oasis.novia.net says...
  15. >
  16. >
  17. > START MESSAGE:
  18. > 
  19. > 
  20. >  I have a question on C... why isn't this program working?!
  21. >  
  22. >  Program START:
  23. >  /* File Open */
  24. >  
  25. >  #include <clib/dos_protos.h>
  26. >  #include <dos/dos.h>
  27. >  #include <stdio.h>
  28. >  #include <exec/types.h>
  29. >  
  30. >  void main();
  31. >  
  32. >    char *name;
  33. >    int age;
  34. >            
  35. >  void main()
  36.  
  37. *** First watch out for the ANSI/ISO police on typing "main" to not 
  38. *** return a value. You might try "int main()".
  39.  
  40. >  
  41. >  {
  42. >    struct FileHandle *file_handle;
  43. >    long bytes_written;
  44. >    long bytes_read;
  45. >  
  46. >   printf("Enter your name: ");
  47. >   scanf("%s",name);
  48.  
  49. *** Second, you are reading input into an undefined area of memory
  50. *** in that you have not set "*name" to point to anything. You could
  51. *** be overwriting anything at this point. Very dangerous! You should
  52. *** be using a buffer, not an undefined pointer. For example,
  53. ***      char name[80];
  54.  
  55. >   printf("\nEnter your age: ");  scanf("%d",age);
  56.  
  57. *** Third, (you're new to 'C', aren't you?) you must precede variables
  58. *** in a "scanf" with the "&" operator to tell "scanf" where to put
  59. *** the input value. This does not,however, apply to arrays and pointers,
  60. *** but you're using an "int" so you need to do it. That is, recode
  61. *** "scanf("%d",age);" as "scanf("%d",&age);". The way you have coded it
  62. *** you are again overwriting memory somewhere, but you are not putting
  63. *** a value in "age" (more than likely). You are putting a value in the
  64. *** memory address that corresponds to the value integer value contained
  65. *** in "age", not in "age" itself.
  66.  
  67. *** My eyes are getting tired, so I'll leave the rest for somebody else.
  68. *** Believe me, there is enough here to keep you busy for awhile.
  69.  
  70. >  ... Snipped ...
  71.  
  72. Hope this helps,
  73. Greg DiGiorgiop
  74.  
  75.